feat(cli): mieweb target via @mieweb/os-cloud-provider by runleveldev · Pull Request #15 · mieweb/cloud

runleveldev · 2026-10-01T14:05:33Z

Wires the opensource-server provider (@mieweb/os-cloud-provider, from mieweb/opensource-server#484, released to npmjs as 2026.10.3) into @mieweb/cli. Rebased onto main after #14 merged, and squashed into one commit.

Changes

--target mieweb uses the built-in provider @mieweb/os-cloud-provider@^2026.10.3, a regular dependency installed from npmjs. deploy, destroy, tail, login, logout and whoami go through it. Its default image is the matching release, ghcr.io/mieweb/opensource-server/cloud:v2026.10.3.
Fallback: commands the provider doesn't implement (dev) run on the Node host harness instead of erroring with "does not implement".
DeployContext.persistTargetConfig: a provider can save settings it resolved interactively to mieweb.jsonc. For example, the provider prompts for an os.mieweb.org site and saves it. The CLI edits the file with jsonc-parser, keeping comments, and refuses secret-looking keys.
mieweb init adds "start": "mieweb --target mieweb dev", which the cloud image's app.service runs. The host harness listens on $PORT when it's set.
Removed pr-preview.yml: it published PR builds to GitHub Packages for testing across repos while the contract and provider were unreleased. Both are now on npmjs.
Tests for provider resolution, the fallback and write-back; README, config schema and changesets updated.

Verified

CI-equivalent run locally: typecheck, all tests, and pnpm check:packs, which confirms no workspace: ranges in the packed tarballs.
Clean install: the CLI was packed with pnpm pack and installed into an empty project with npm i <tgz> --userconfig /dev/null (npmjs only, no token). The provider 2026.10.3 and @mieweb/*@0.3.1 all resolved from npmjs.
- Without credentials, mieweb whoami returns "Not authenticated — provider opensource-server", and deploy with a bad token gives the login hint.
- With a real API key, whoami against manager.os.mieweb.org returns "Authenticated as rgingras".
A full deploy → browser check → tail → redeploy → destroy run against manager.os.mieweb.org was done earlier with the preview builds of this code; see the PR history.

Also fixed (folded in from #18)

Missing wrangler: with wrangler not installed, wrangler-delegated commands on the cloudflare target used to end in a stack trace (a regression from feat(deploy): provider contract + wrangler reference provider #14). They now print mieweb: wrangler binary not found… and exit 1.
mieweb --version: prints the mieweb version and exits 0 on every target. Before, it printed an error after the version on non-cloudflare targets.
Three tests; patch changeset cli-missing-wrangler.

- built-in provider for --target mieweb: @mieweb/os-cloud-provider ^2026.10.3 (npmjs) - verbs a provider doesn't implement (dev) fall back to the Node host harness - DeployContext.persistTargetConfig: write resolved settings (e.g. the picked site) back into mieweb.jsonc, keeping comments (jsonc-parser) - mieweb init adds a start script for the cloud image; the host harness honors $PORT - remove the GitHub Packages preview workflow (cross-repo previews are no longer needed)

…rks without it

…flare targets

Copilot

🟡 Changes recommended

Secret validation, optional-auth routing, scaffold dependencies, and Node compatibility remain unresolved.

5 open issues

1 file not reviewed

pnpm-lock.yaml: Generated file

What changed in this PR

Integrates the opensource-server deploy provider into the CLI’s mieweb target while retaining the Node host harness for development.

Changes:

Bundles the provider, adds optional-command fallback, and enables config write-back.
Adds scaffolded startup support and honors $PORT.
Improves missing-wrangler/version handling and removes preview publishing.

File	Description
pnpm-lock.yaml	Locks provider dependencies.
packages/cloud-adapters/src/host.mjs	Honors `$PORT`.
packages/cli/test/provider.test.mjs	Tests provider resolution and config persistence.
packages/cli/test/provider-core.test.mjs	Tests missing wrangler and version output.
packages/cli/src/provider.mjs	Adds provider mapping and config write-back.
packages/cli/src/init.mjs	Generates a start script.
packages/cli/src/index.mjs	Updates fallback routing and version handling.
packages/cli/src/cloudflare.mjs	Reports wrangler execution failures cleanly.
packages/cli/README.md	Documents provider-backed commands.
packages/cli/package.json	Adds the bundled provider dependency.
packages/cli/mieweb-config.schema.json	Documents provider defaults and fallback.
.github/workflows/pr-preview.yml	Removes preview publishing.
.changeset/init-start-script.md	Records startup and port support.
.changeset/cli-os-cloud-provider.md	Records provider integration and persistence.
.changeset/cli-missing-wrangler.md	Records wrangler and version fixes.

_{Review effort: Balanced}

- engines: node >=22.18 (required by @mieweb/os-cloud-provider) - only dev/tail fall back to the legacy path when a provider lacks them; login/logout/whoami/destroy keep runProviderVerb's explanations - mieweb init: depend on this CLI's release and on @libsql/client + ioredis, which the generated mieweb bindings (and npm start on the cloud image) need - persistTargetConfig: refuse secret-looking keys anywhere in the patch, including nested objects/arrays and access-key names

Copilot

🔵 Needs a closer look

The generated start script depends on a CLI omitted by production-only installs.

No open issues 🎉

1 previously missed issue

In code that hasn't changed since last review

Declare @mieweb/cli as a production dependency

packages/cli/src/init.mjs:119

The new start script invokes mieweb at runtime, but the scaffold declares @mieweb/cli only as a devDependency. A production-only install (npm ci --omit=dev) therefore omits the executable required by npm start, causing startup to fail with mieweb: not found. Move @mieweb/cli into dependencies and update the scaffold test to check its runtime dependency entry.

5 resolved since last review

1 file not reviewed

pnpm-lock.yaml: Generated file

_{Review effort: Balanced}

…dependencies (npm start runs mieweb)

Copilot

🟢 Approval recommended

The changes address prior feedback, include regression coverage for the main integration paths, and have no unresolved review findings.

No open issues 🎉

1 file not reviewed

pnpm-lock.yaml: Generated file

_{Review effort: Balanced}

runleveldev force-pushed the cli-os-cloud-provider branch 5 times, most recently from 811311a to 60d51d9 Compare October 7, 2026 17:36

runleveldev force-pushed the cli-os-cloud-provider branch from 60d51d9 to 1986519 Compare October 20, 2026 14:52

runleveldev changed the base branch from deploy-provider-contract to main October 20, 2026 14:52

runleveldev added 2 commits October 20, 2026 10:54

fix(cli): report a missing wrangler instead of crashing; --version wo…

d901095

…rks without it

fix(cli): mieweb --version exits 0 with just the version on non-cloud…

522247e

…flare targets

runleveldev mentioned this pull request Oct 20, 2026

fix(cli): report a missing wrangler instead of crashing #18

Closed

runleveldev marked this pull request as ready for review October 20, 2026 15:08

Copilot AI balanced review requested due to automatic review settings October 20, 2026 15:08

Copilot started reviewing on behalf of runleveldev October 20, 2026 15:08

View session

Copilot AI reviewed Oct 20, 2026

View reviewed changes

Comment thread packages/cli/package.json Resolved

Comment thread packages/cli/src/index.mjs Resolved

Comment thread packages/cli/src/init.mjs Resolved

Comment thread packages/cli/src/init.mjs Resolved

Comment thread packages/cli/src/provider.mjs Resolved

Copilot AI balanced review requested due to automatic review settings October 20, 2026 15:24

Copilot started reviewing on behalf of runleveldev October 20, 2026 15:25

View session

Copilot AI reviewed Oct 20, 2026

View reviewed changes

fix(cli): mieweb init lists @mieweb/cli and @mieweb/cloud as runtime …

1fae403

…dependencies (npm start runs mieweb)

Copilot AI balanced review requested due to automatic review settings October 20, 2026 15:34

Copilot started reviewing on behalf of runleveldev October 20, 2026 15:35

View session

Copilot AI reviewed Oct 20, 2026

View reviewed changes

Navigation Menu

Repository navigation

feat(cli): mieweb target via @mieweb/os-cloud-provider - #15

feat(cli): mieweb target via @mieweb/os-cloud-provider#15
runleveldev wants to merge 5 commits into
mainfrom
cli-os-cloud-provider

runleveldev commented Oct 1, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

runleveldev commented Oct 1, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

Verified

Also fixed (folded in from #18)

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟡 Changes recommended

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🔵 Needs a closer look

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

🟢 Approval recommended

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

runleveldev commented Oct 1, 2026 •

edited

Loading